Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add writeBlockData method #2

Merged
merged 1 commit into from Dec 30, 2013

Conversation

jonathanstowe
Copy link
Contributor

Hi,
Thanks for making Device::SMBus.

I have used it to make Device::PiGlow and it works great.

I noticed that you implemented the XS code for the block write but didn't expose it in the Perl API, this implements that.

I've actually implemented it directly in my code, and it works fine as implemented (for what I wanted to do.) So know rush to make a release :)

Because I don't have a device that I can read from I didn't implement the read block data :)

shantanubhadoria added a commit that referenced this pull request Dec 30, 2013
@shantanubhadoria shantanubhadoria merged commit 2300b56 into shantanubhadoria:master Dec 30, 2013
@shantanubhadoria
Copy link
Owner

Thanks for this addition.
I will put it into the next release.
Cheers,
-Shantanu Bhadoria

shantanubhadoria added a commit that referenced this pull request Dec 30, 2013
 - spelling fixes [Shantanu Bhadoria]
 - Added more documentation, used mailmap to remove duplicates in
   contributors information, Thanks to git shortlog:
   https://www.kernel.org/pub/software/scm/git/docs/git-shortlog.html
   [Shantanu Bhadoria]
 - Merge pull request #2 from jonathanstowe/master

   add writeBlockData method [Shantanu Bhadoria]
 - add writeBlockData method [Jonathan Stowe]
@jonathanstowe
Copy link
Contributor Author

:)
Thanks for making a useful module.

On Sun, 2013-12-29 at 20:44 -0800, Shantanu Bhadoria wrote:

Thanks for this addition.
I will put it into the next release.
Cheers,
-Shantanu Bhadoria


Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants